Skip to content

sync with latest GIE after some changes to the scheduling plugins#197

Merged
nirrozenbaum merged 2 commits intollm-d:mainfrom
nirrozenbaum:sync-gie
Jun 24, 2025
Merged

sync with latest GIE after some changes to the scheduling plugins#197
nirrozenbaum merged 2 commits intollm-d:mainfrom
nirrozenbaum:sync-gie

Conversation

@nirrozenbaum
Copy link
Collaborator

This PR updates llm-d-scheduler plugins after some changes were made to interfaces in GIE upstream.
more concretely, the following PRs were merged in GIE:

the last one was also considered as a potential improvement that might help in #189 (cc @kfirtoledo).
once this PR merges, Kfir can explore moving some P/D specifics out of the prefix scorer and into the ProcessResults function of the PDProfileHandler.

Signed-off-by: Nir Rozenbaum <nirro@il.ibm.com>
Signed-off-by: Nir Rozenbaum <nirro@il.ibm.com>
Copy link
Collaborator

@shmuelk shmuelk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

/approve

@nirrozenbaum nirrozenbaum merged commit 437f887 into llm-d:main Jun 24, 2025
2 of 3 checks passed
@nirrozenbaum nirrozenbaum deleted the sync-gie branch June 24, 2025 11:50
Jooho pushed a commit to Jooho/llm-d-inference-scheduler that referenced this pull request Sep 30, 2025
Signed-off-by: konflux-internal-p02 <170854209+konflux-internal-p02[bot]@users.noreply.github.com>
Co-authored-by: konflux-internal-p02[bot] <170854209+konflux-internal-p02[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants